From: Marco Trevisan (TreviƱo) Date: Tue, 16 Feb 2021 02:45:14 +0000 (+0100) Subject: gdm: Filter service non-error messages on verification stopped or failed X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~616^2^2~320 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=45a5171a95c7337a541c765306b5e5fd7f790de3;p=gnome-shell.git gdm: Filter service non-error messages on verification stopped or failed Once the verification has been stopped or has failed all the messages that are not of error type are just not needed or wrong to show. For example, in the fingerprint case we may still show the hint to swipe or touch the device, while the fingerprint PAM service has already been stopped. So filter them by adding a new function that adds a null message to the queue, overriding all the messages that have a lower priority. Part-of: --- diff --git a/js/gdm/util.js b/js/gdm/util.js index 67255bc9e..f1c3c9149 100644 --- a/js/gdm/util.js +++ b/js/gdm/util.js @@ -281,6 +281,9 @@ var ShellUserVerifier = class { } _getIntervalForMessage(message) { + if (!message) + return 0; + // We probably could be smarter here return message.length * USER_READ_TIME; } @@ -299,6 +302,18 @@ var ShellUserVerifier = class { this._currentMessageExtraInterval = interval; } + _serviceHasPendingMessages(serviceName) { + return this._messageQueue.some(m => m.serviceName === serviceName); + } + + _filterServiceMessages(serviceName, messageType) { + // This function allows to remove queued messages for the @serviceName + // whose type has lower priority than @messageType, replacing them + // with a null message that will lead to clearing the prompt once done. + if (this._serviceHasPendingMessages(serviceName)) + this._queuePriorityMessage(serviceName, null, messageType); + } + _queueMessageTimeout() { if (this._messageQueueTimeoutId != 0) return; @@ -678,6 +693,7 @@ var ShellUserVerifier = class { const canRetry = retry && this._canRetry(); this._disconnectSignals(); + this._filterServiceMessages(serviceName, MessageType.ERROR); if (canRetry) { if (!this.hasPendingMessages) { @@ -730,6 +746,8 @@ var ShellUserVerifier = class { return; } + this._filterServiceMessages(serviceName, MessageType.ERROR); + if (this._unavailableServices.has(serviceName)) return;